Skip to content

Fix: add GH_TOKEN to trigger workflow detect step#1228

Merged
pawel-kow merged 1 commit into
masterfrom
feature/add-labels-from-PR-review
Jun 14, 2026
Merged

Fix: add GH_TOKEN to trigger workflow detect step#1228
pawel-kow merged 1 commit into
masterfrom
feature/add-labels-from-PR-review

Conversation

@pawel-kow

@pawel-kow pawel-kow commented Jun 14, 2026

Copy link
Copy Markdown
Member

Fix: GH_TOKEN missing in trigger workflow

Problem

The detect step in review-trigger.yml calls gh api to check whether a
push is an "Update branch" merge commit, but GH_TOKEN was not set in the
step's environment. This caused the workflow to exit with code 4:

gh: To use GitHub CLI in a GitHub Actions workflow, set the GH_TOKEN environment variable.

Fix

Added GH_TOKEN: ${{ github.token }} to the detect step's env and
permissions: contents: read at the job level (required for the commits API).

@pawel-kow pawel-kow merged commit ad0df94 into master Jun 14, 2026
3 of 5 checks passed
@pawel-kow pawel-kow deleted the feature/add-labels-from-PR-review branch June 14, 2026 11:01
@github-actions github-actions Bot added Checklist of common problems not complete See PR template and mark *all* checkboxes, even if not applicable. Explain any discrepancies. PR description incomplete The PR description template was not filled in at all, altered or filled in improperly. Test links missing No test links from Online Editor provided labels Jun 14, 2026
@github-actions

github-actions Bot commented Jun 14, 2026

Copy link
Copy Markdown

❌ JSON Filename Check Failed

  • File in subdirectory not allowed: .github/workflows/review-trigger.yml

@github-actions

github-actions Bot commented Jun 14, 2026

Copy link
Copy Markdown

PR Description Check Failed

The PR description is missing required elements. Please update it according to the PR template.

Details 

Labels to add: Checklist of common problems not complete, PR description incomplete, Test links missing
Labels to remove: Forged editor links

PR description check FAILED:
  FAIL  'Type of change' section not found
  FAIL  'How Has This Been Tested?' section not found
  FAIL  'Checklist of common problems' section not found
  FAIL  'Online Editor test results' section not found

@github-actions github-actions Bot added the filename-error Files not in main folder or not named correctly label Jun 14, 2026
@github-actions

github-actions Bot commented Jun 14, 2026

Copy link
Copy Markdown

✅ JSON Schema Validation Passed

@github-actions

github-actions Bot commented Jun 14, 2026

Copy link
Copy Markdown

Linter OK

@github-actions github-actions Bot added the automerge-possible Label indicating, that restrictive linter checks all pass. For now it's PoC. label Jun 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

automerge-possible Label indicating, that restrictive linter checks all pass. For now it's PoC. Checklist of common problems not complete See PR template and mark *all* checkboxes, even if not applicable. Explain any discrepancies. filename-error Files not in main folder or not named correctly PR description incomplete The PR description template was not filled in at all, altered or filled in improperly. Test links missing No test links from Online Editor provided

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@pawel-kow